Skip to content

[6.0] Update layouts to use Form control fields#45694

Merged
softforge merged 37 commits intojoomla:6.0-devfrom
Fedik:form-control-fields-com1
Aug 6, 2025
Merged

[6.0] Update layouts to use Form control fields#45694
softforge merged 37 commits intojoomla:6.0-devfrom
Fedik:form-control-fields-com1

Conversation

@Fedik
Copy link
Member

@Fedik Fedik commented Jul 6, 2025

Summary of Changes

Follow up for:

The PR changes layouts to use Form control fields API.

Updated following forms and list views:

com_actionlogs
com_associations
com_banners
com_cache
com_checkin
com_config
com_contact
com_fields
com_finder
com_guidedtours
com_installer
com_languages
com_mails
com_messages
com_modules
com_newsfeeds
com_plugins
com_privacy
com_redirect
com_scheduler
com_tags
com_templates
com_users
com_workflow

Testing Instructions

There is no visible changes.

However can tests a few of the modified components from the list above, and make sure all works as before.
Add/edit/delete its content.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:
  • No documentation changes for docs.joomla.org needed
  • Pull Request link for manual.joomla.org: Form control fields new feature Manual#322
  • No documentation changes for manual.joomla.org needed

…s/HtmlView.php

Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
@exlemor
Copy link

exlemor commented Jul 11, 2025

I have tested this item ✅ successfully on 59e0991

I have tested this succesfully.

I have specifically tested:

com_banners
com_cache
com_checkin
com_config
com_contact
com_guidedtours
com_languages
com_newsfeeds
com_scheduler
com_tags
com_users


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45694.

@chmst
Copy link
Contributor

chmst commented Jul 11, 2025

I have tested this item ✅ successfully on 59e0991

Tested some randomly chosen views, no errores found


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45694.

@QuyTon
Copy link
Contributor

QuyTon commented Jul 11, 2025

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45694.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 11, 2025
@muhme
Copy link
Contributor

muhme commented Jul 11, 2025

I have tested this item ✅ successfully on 59e0991

Tested with JBT from pre-build Joomla_6.0.0-alpha3-dev+pr.45694-Development-Full_Package.zip

  • com_banners: created, listes, batch copied
  • com_contact: created, listed, all fields used, batch copied + moved
  • com_users: created, all fields used, batch group assigned, deleted
  • com_plugins: enabled, disabled, list view, and pulgin detail view
  • com_modules: installed zitat-service.de from JED configured and tested on frontpage
    This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45694.

@richard67
Copy link
Member

It seems this PR introduces new phpstan errors, see the phpstan notices below the changes in the changed files of this PR. The errors should either be fixed (if possible without b/c break) or be added to the phpstan-baseline.neon file. The easiest way to to the latter is to run composer install and then ./libraries/vendor/bin/phpstan -b to update the phpstan-baseline.neon file. That can also be done after some of the errors were fixed so only the remaining errors are added to the file.

@softforge softforge added this to the Joomla! 6.0.0 milestone Aug 6, 2025
@softforge softforge merged commit ddfe0d2 into joomla:6.0-dev Aug 6, 2025
34 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 6, 2025
@softforge
Copy link
Contributor

thank you @Fedik and all the testers and reviewers who put the time in

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants